Conversation
Final sweep before v0.4.0. Things found that the per-PR cleanup
didn't catch:
main/index.ts: drop the spool:// custom protocol registration. The
URL handler (parseSpoolUrl + handleSpoolUrl) was removed in PR2,
leaving the registration silently broken — clicking spool://… URLs
would do nothing. Cleaner to not register a scheme we don't handle.
landing/pages/docs/guides/data-sources.md: rewrite to be sessions-
only; replace the "Platform data (via connector plugins)" section
with a forward to Spool Daemon. The published doc currently tells
users to run `spool connectors install …`, which exits 1 on the
shipped CLI.
landing/pages/docs/quick-start.md: drop step 3 ("Install connector
plugins"); replace with the actual CLI commands and a forward to
Spool Daemon at the end.
landing/pages/docs/reference/configuration.md: drop the
~/.spool/connectors/ row from the data-directory table; add ui.json
in its place.
landing/pages/docs/guides/agent-integration.md: drop "bookmarks,
stars" from the planned-features list; drop "connector data" from
the local-search-index description.
landing/pages/blog/hello-spool.md: rewrite the connector-plugin
paragraph as a forward to Spool Daemon for platform data.
Verified app still builds clean (electron-vite + tsc). Landing has
no build step in this monorepo (consumed via /daemon redirect +
docs static rendering by the landing's own deploy).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
graydawnc
changed the title
graydawnc
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Final sweep before v0.4.0 — stale references the per-feature M5 PRs (#112–#117) didn't catch.
Changes
packages/app/src/main/index.tsspool://custom-protocol registrationpackages/landing/pages/docs/guides/data-sources.mdspool connectors install …, which exits 1 on the shipped CLIpackages/landing/pages/docs/quick-start.mdpackages/landing/pages/docs/reference/configuration.md~/.spool/connectors/row; addui.jsonrowpackages/landing/pages/docs/guides/agent-integration.mdpackages/landing/pages/blog/hello-spool.mdNotes
~/.spool/connectors/left over on existing user disks is harmless (a few KB of stale node_modules from old connector installs); not worth a startup cleanup pass.docs/spool-v0.png) was not updated — its content (search box) is product-direction-stable. Refresh later if it ever shows the removed UI tabs.docs/superpowers/*historical plan/spec records are git-log-style artifacts and intentionally left intact.Test plan
pnpm exec tsc --noEmitclean (app)pnpm build:electronpasses (app bundle unchanged in size after dropping a handful of bytes of registration code)grep -r 'spool connectors\|spool connector ' --include='*.md' --include='*.tsx' --include='*.ts' --include='*.json' --include='*.yml'returns zero outside historical superpowers archives🤖 Generated with Claude Code